Skip to content

Conversation

@tobio
Copy link
Member

@tobio tobio commented Sep 22, 2025

Fixes #520

This resource only manages the existence of an anomaly detection job, it does not manage the state (open/closed). I'm planning on adding another resource to purely manage that state, it's an independent API. IMO it's simpler for practitioners if these are separate resources altogether.

@tobio tobio requested review from dimuon and nick-benoit September 22, 2025 05:45
@tobio tobio self-assigned this Sep 22, 2025
@tobio tobio mentioned this pull request Sep 28, 2025
@tobio tobio mentioned this pull request Sep 29, 2025
Copy link
Contributor

@nick-benoit nick-benoit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Just a few questions.

@tobio tobio requested a review from nick-benoit October 21, 2025 10:19
nick-benoit
nick-benoit previously approved these changes Oct 21, 2025
Copy link
Contributor

@nick-benoit nick-benoit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@tobio tobio enabled auto-merge (squash) October 23, 2025 11:10
@tobio tobio disabled auto-merge October 23, 2025 11:18
@tobio tobio merged commit d1eb314 into main Oct 23, 2025
29 checks passed
@tobio tobio deleted the ml-anomaly branch October 23, 2025 11:18
tobio added a commit that referenced this pull request Oct 23, 2025
* origin/main:
  Add resource to manage ML anomaly detection jobs (#1329)
  chore(deps): pin alpine/mkcert docker tag to a8f4f5a (#1389)
  Bump version in Makefile (#1388)
  Update changelog for 0.11.19 and 0.12.1 (#1387)
  Security Detection Rule schema validation improvements (#1381)
  Remove connector client generation make task (#1386)
  Fixup changelog link
  Move local environment to docker compose managed services (#1384)
  chore(deps): update golang:1.25.3 docker digest to 8c945d3 (#1385)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] Anomaly Detection Jobs

3 participants